[자동차 경주] 염수환 미션 제출합니다.#56
Conversation
todays-sun-day
left a comment
todays-sun-day
left a comment
There was a problem hiding this comment.
코드 잘 보고 갑니다!
다음 과제 때는 테스트 코드도 있으면 더욱 좋을 것 같아요!
2주차 고생많으셨어요. 3주차도 화이팅입니다 !!
There was a problem hiding this comment.
커밋 단위가 크다고 여겨집니다! 한 기능에 한 커밋씩으로 나눠보면 어떨까요?
깜빡하고 커밋을 하지 않고 많은 구현을 하셨을 때 fork 같은 깃 클라이언트를 사용하면, 다시 파일별로 선택해서 커밋하실 수도 있어요!
There was a problem hiding this comment.
저도 수환님과 같은 형태로 인풋뷰를 작성했는데요. 출력과 입력을 나누면 어떠할지에 대한 리뷰를 받았어요!
확실히 보고 쓰기엔 편한 것처럼 느껴지는데, 기능 분리에는 소홀해서 나눠보는 것도 고려해보시면 좋겠습니다!
joon0447
left a comment
joon0447
left a comment
There was a problem hiding this comment.
안녕하세요! 2주차 과제 고생 많으셨습니다!
@todays-sun-day 님도 말씀해주셨지만, 커밋 단위가 너무 큰 거 같아요!
작게 쪼개어 커밋하면 코드리뷰하는데에도 많은 도움이 될 것 같습니다
| val number = Randoms.pickNumberInRange(0, 9) | ||
| return number >= 4 |
There was a problem hiding this comment.
공통피드백에도 나와있지만, 0과 9, 그리고 4를 상수화하였으면 더 가독성 좋은 코드가 되었을 것 같아요!
| object InputView { | ||
| fun inputCarNames(): String { |
There was a problem hiding this comment.
InputView는 있는데, 따로 OutputView를 안 만드신 이유가 있으실까요?
aksworns22
left a comment
aksworns22
left a comment
There was a problem hiding this comment.
안녕하세요 잘 읽었습니다!
파일 끝에 빨간 동그라미가 뜨는 것과 관련해서 Discord에 공유된 글입니다!
저도 방금 알게 된 내용인데 한번 확인해보시면 도움이 될 것 같습니다 😊
| fun move(rule: MoveRule) { | ||
| if (rule.canMove()) position++ | ||
| } |
There was a problem hiding this comment.
MoveRule이라는 규칙에 대한 객체를 사용하시는 모습이 인상깊었습니다 😮
현재 MoveRule은 이동 여부(canMove())만 판단하고 있는데 이동 거리까지 함께 결정하도록 확장해봐도 좋을 것 같습니다!
4 participants
No description provided.